Hey! Your PR title Fix: Roslyn crash via incremental sync range and broken workspaceSymbol query doesn't follow conventional commit format.

Please update it to start with one of:

• feat: or feat(scope): new feature
• fix: or fix(scope): bug fix
• docs: or docs(scope): documentation changes
• chore: or chore(scope): maintenance tasks
• refactor: or refactor(scope): code refactoring
• test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

I did this recently #23771

@VladD2 are you able to link me a repo/repro that fails on the latest opencode, but passes on this PR?

I did this recently #23771

@VladD2 are you able to link me a repo/repro that fails on the latest opencode, but passes on this PR?

Ah, my changes to packages/opencode/src/lsp/client.ts were made before #23771 landed — I've reverted those.

The core of this PR is LSP method parameter validation and fixs:

• workspaceSymbol now passes the query from the AI to the LSP (previously always sent empty string, which was the main bug)
• workspaceSymbol now requires a query parameter (previously always sent as empty string, causing LSP servers to return all symbols)
• documentSymbol only requires filePath
• All other operations require filePath, line, and character

PS

Please review this PR soon — I'm getting merge conflicts with dev every day.

@Hona @rekram1-node Please review this PR soon — I'm getting merge conflicts with dev every day.